PlanetaryComputer SDK for Management Plane #50835
Open
+32,924
−8,149
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shaktichetan23
requested review from
ArcturusZhang,
ArthurMa1978 and
jsquire
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR introduces the PlanetaryComputer management plane SDK for .NET, including project structure, CI pipelines, generated APIs, and tests.
- Add new
Azure.ResourceManager.PlanetaryComputerclient library with full CRUD support forGeoCatalogresources - Configure CI triggers and pipeline YAML for the new package
- Include recorded-playback tests, samples, documentation, and versioning files
Reviewed Changes
Copilot reviewed 19 out of 56 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/resourcemanager/ci.mgmt.yml | Include PlanetaryComputer package in CI triggers |
| sdk/planetarycomputer/ci.mgmt.yml | Add CI configuration for PlanetaryComputer SDK |
| sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/src/… | Generate API, project files, and assembly info |
| sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/tests/… | Add recorded-playback tests and environment setup |
| sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/README.md | Provide README with usage and testing instructions |
Comments suppressed due to low confidence (3)
sdk/planetarycomputer/ci.mgmt.yml:16
- Stray lines and misplaced slashes under
paths.includewill break YAML parsing; remove or properly indent these lines to match the intended file paths.
/ci.mgmt.yml
sdk/planetarycomputer/ci.mgmt.yml:13
- Path casing mismatch: the actual directory is
sdk/planetarycomputer, so this include won’t match the file system. Change to lowercaseplanetarycomputer.
- sdk/PlanetaryComputer/ci.mgmt.yml
sdk/resourcemanager/ci.mgmt.yml:162
- Trigger path uses
PlanetaryComputercasing, but the folder name isplanetarycomputer; adjust casing so the CI picks up changes.
- sdk/PlanetaryComputer/Azure.ResourceManager.PlanetaryComputer
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/src/Properties/AssemblyInfo.cs
Outdated
Show resolved
Hide resolved
...computer/Azure.ResourceManager.PlanetaryComputer/tests/Scenario/GeoCatalogCollectionTests.cs
Outdated
Show resolved
Hide resolved
…src/Properties/AssemblyInfo.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…tests/Scenario/GeoCatalogCollectionTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
…o planetarycomputer_dotnet_sdk
…re/azure-sdk-for-net into planetarycomputer_dotnet_sdk
* feat: add support for the 2024-02-01 metric ns API * Update sdk/monitor/Azure.Monitor.Query/CHANGELOG.md Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com> --------- Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com>
* Fix issue with next link paging * fix * regen
ArthurMa1978
requested changes
Jun 25, 2025
There was a problem hiding this comment.
Please make sure you are member of the Azure and Microsoft GitHub organization, for more details please review the Azure SDK onboarding documentation and use the associated Teams channel for support.
- Azure SDK onboarding (Microsoft internal)
- Azure SDK onboarding assistance (Microsoft internal)
You can verify the state of your account by running the Validate-AzsdkCodeOwner script from the Azure SDK tools repository.
Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.
…roperties are hidden in mgmt libraries (#50814) * finish appservice * finish container service * finish postgresql * finish storage * generator changes * revert api-version changes on storage * add changelogs
* Increment package version after release of Azure.Provisioning.Storage * Increment package version after release of Azure.Provisioning.PostgreSql
* Increment package version after release of Azure.Provisioning.ContainerService * Increment package version after release of Azure.Provisioning.AppService
shaktichetan23
requested review from
vicancy,
JialinXin,
KrzysztofCwalina,
seanmcc-msft,
amnguye,
jaschrep-msft,
jalauzon-msft,
nickliu-msft,
ShivangiReja,
a team,
m-redding,
LuChen-Microsoft,
MikeyMCZ,
joseharriaga,
vikaspalaskar,
quentinRobinson,
bidisha-c,
trangevi,
dargilco,
jhakulin,
glharper and
nick863
as code owners
shaktichetan23
requested review from
joshfree,
pvaneck,
KevinBlasko,
KarishmaGhiya,
scottaddie,
srnagar and
jairmyree
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.